Skip to content

Conversation

@smaslov-intel
Copy link
Contributor

The expectation is that this might improve performance since no redundant cache flushes occur on the wait-side.
Signed-off-by: Sergey V Maslov sergey.v.maslov@intel.com

Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
@smaslov-intel
Copy link
Contributor Author

/summary:run

@smaslov-intel smaslov-intel requested a review from bso-intel March 26, 2021 15:36
Copy link
Contributor

@bso-intel bso-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smaslov-intel smaslov-intel changed the title [SYCL] Avoid HOST .wait scope; .signal=HOST is expected to be enough [SYCL][DO NOT MERGE] Avoid HOST .wait scope; .signal=HOST is expected to be enough Mar 27, 2021
@bader
Copy link
Contributor

bader commented Mar 29, 2021

image

I suggest converting to Draft instead of adding [DO NOT MERGE] tag. GitHub UI doesn't allow merging a draft automatically.

@smaslov-intel smaslov-intel changed the title [SYCL][DO NOT MERGE] Avoid HOST .wait scope; .signal=HOST is expected to be enough [SYCL] Avoid HOST .wait scope; .signal=HOST is expected to be enough Mar 29, 2021
@smaslov-intel
Copy link
Contributor Author

It's ready to go in.

@bader bader merged commit 77fa03f into intel:sycl Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants